Skip to content

Conversation

@malayladu
Copy link
Contributor

@malayladu malayladu commented Feb 6, 2025

Context

⛑️ Ticket(s): https://secure.helpscout.net/conversation/2828248311/77058

Summary

Added support for handling the meta modifier to display meta key values.

This update enhances the All Fields Template plugin by enabling support for the :meta[] entry meta to display values of meta keys.

One can use {all_fields} merge tags with meta modifier like

{all_fields:meta[gpev_basic_status_3,gppcmt_pretty_id]}

Here's the loom

https://www.loom.com/share/3f40a38e40c74950bc93c74a03006389?sid=b514a59f-b909-45d7-ab85-3bb9d5712b1c

@malayladu malayladu self-assigned this Feb 6, 2025
@github-actions
Copy link

github-actions bot commented Feb 6, 2025

Fails
🚫

Pull request title does match the correct format. The Pull Request title should match our Snippet Library Pull Request Title Guidelines in Notion.

🚫

Commit message 'gw-all-fields-template.php: Added support to handle meta modifier to display meta keys values.' does match the correct format. See our Snippet Library Commit Messages Guidelines in Notion.

Generated by 🚫 dangerJS against db29b8f

@malayladu malayladu changed the title gw-all-fields-template.php: Added support to handler meta modifier to display meta keys values gw-all-fields-template.php: Added support to handler meta modifier to display meta keys values. Feb 6, 2025
@malayladu malayladu changed the title gw-all-fields-template.php: Added support to handler meta modifier to display meta keys values. gw-all-fields-template.php: Added support to handle meta modifier to display meta keys values. Feb 6, 2025
@malayladu malayladu force-pushed the malay/add/77058-show-entry-meta branch 2 times, most recently from 16b11a2 to 4142018 Compare February 6, 2025 11:33
@malayladu malayladu force-pushed the malay/add/77058-show-entry-meta branch from 4142018 to db29b8f Compare February 6, 2025 12:03
@spivurno spivurno removed their request for review February 6, 2025 12:24
Copy link
Contributor

@saifsultanc saifsultanc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working well! Sending for final confirm to Clay.

Copy link
Contributor

@claygriffiths claygriffiths left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! S&M

@claygriffiths claygriffiths merged commit bee28e2 into master Feb 11, 2025
4 of 6 checks passed
@claygriffiths claygriffiths deleted the malay/add/77058-show-entry-meta branch February 11, 2025 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants